Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add contract events #269

Merged
merged 22 commits into from
Nov 26, 2024
Merged

Add contract events #269

merged 22 commits into from
Nov 26, 2024

Conversation

DOBEN
Copy link
Member

@DOBEN DOBEN commented Oct 15, 2024 •
edited
Loading

Purpose

Addresses #253

Changes

  • Add contractEvents fields.
  • Add logic to decode input_parameter/message if an embedded smart contract schema exists.
  • Add logic to decode contract logs if an embedded smart contract schema exists.

@DOBEN DOBEN changed the base branch from main to rust-backend-modules October 15, 2024 14:59
Base automatically changed from rust-backend-modules to rust-backend October 21, 2024 17:31
Base automatically changed from rust-backend to main October 24, 2024 09:51
@DOBEN DOBEN force-pushed the add-contract-module-events branch 2 times, most recently from de04c9f to 8ea3d7e Compare October 30, 2024 13:18
Victor-N-Suadicani
Victor-N-Suadicani approved these changes Oct 31, 2024
View reviewed changes
Copy link
Contributor

@Victor-N-Suadicani Victor-N-Suadicani left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Various stylistic comments and suggestions, nothing blocking.

backend-rust/migrations/0001_initialize.up.sql Show resolved Hide resolved
backend-rust/src/graphql_api.rs Outdated Show resolved Hide resolved
backend-rust/src/graphql_api.rs Outdated Show resolved Hide resolved
backend-rust/src/graphql_api.rs Outdated Show resolved Hide resolved
backend-rust/src/graphql_api.rs Outdated Show resolved Hide resolved
backend-rust/src/graphql_api.rs Outdated Show resolved Hide resolved
backend-rust/src/graphql_api.rs Show resolved Hide resolved
backend-rust/src/indexer.rs Outdated Show resolved Hide resolved
backend-rust/src/indexer.rs Outdated Show resolved Hide resolved
backend-rust/src/indexer.rs Outdated Show resolved Hide resolved
limemloh
limemloh reviewed Oct 31, 2024
View reviewed changes
backend-rust/migrations/0001_initialize.up.sql Outdated Show resolved Hide resolved
backend-rust/migrations/0001_initialize.up.sql Outdated Show resolved Hide resolved
backend-rust/src/indexer.rs Outdated Show resolved Hide resolved
backend-rust/src/indexer.rs Outdated Show resolved Hide resolved
backend-rust/src/indexer.rs Outdated Show resolved Hide resolved
backend-rust/src/graphql_api.rs Outdated Show resolved Hide resolved
backend-rust/src/graphql_api.rs Outdated Show resolved Hide resolved
backend-rust/src/graphql_api.rs Outdated Show resolved Hide resolved
backend-rust/migrations/0001_initialize.up.sql Show resolved Hide resolved
backend-rust/src/graphql_api.rs Outdated Show resolved Hide resolved
@DOBEN DOBEN changed the title Add contract and module events Add contract events Nov 21, 2024
@DOBEN DOBEN requested a review from limemloh November 25, 2024 17:53
@DOBEN DOBEN merged commit a223986 into main Nov 26, 2024
3 checks passed
@DOBEN DOBEN deleted the add-contract-module-events branch November 26, 2024 10:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Victor-N-Suadicani Victor-N-Suadicani Victor-N-Suadicani approved these changes

@limemloh limemloh Awaiting requested review from limemloh

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@DOBEN @limemloh @Victor-N-Suadicani